-
Notifications
You must be signed in to change notification settings - Fork 901
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add contact forms template image #1724
Conversation
…src/' <!-- ELLIPSIS_HIDDEN --> > [!IMPORTANT] > Add contact forms image to `TEMPORARY_TEMPLATE_IMAGES` in `TemporaryTemplateImages.ts`. > > - **Behavior**: > - Add `contactFormsImage` import from `@/assets/contact-forms.png` in `TemporaryTemplateImages.ts`. > - Add `wpid_356554024867713766: contactFormsImage` to `TEMPORARY_TEMPLATE_IMAGES` in `TemporaryTemplateImages.ts`. > > <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=Skyvern-AI%2Fskyvern-cloud&utm_source=github&utm_medium=referral)<sup> for 50743c0f21d472be12517f94f81e5903e39b0b79. It will automatically update as commits are pushed.</sup> <!-- ELLIPSIS_HIDDEN -->
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Reviewed everything up to 5fb163f in 11 seconds
More details
- Looked at
13
lines of code in1
files - Skipped
1
files when reviewing. - Skipped posting
4
drafted comments based on config settings.
1. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:2
- Draft comment:
Confirm import path and asset existence for contactFormsImage. - Reason this comment was not posted:
Confidence changes required:0%
None
2. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:7
- Draft comment:
Verify mapping key and usage for contactFormsImage in TEMPORARY_TEMPLATE_IMAGES. - Reason this comment was not posted:
Confidence changes required:0%
None
3. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:2
- Draft comment:
Verify '@assets/contact-forms.png' exists and follows naming conventions. - Reason this comment was not posted:
Confidence changes required:33%
None
4. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:7
- Draft comment:
Confirm key wpid_356554024867713766 correctly maps to contact forms image. - Reason this comment was not posted:
Confidence changes required:0%
None
Workflow ID: wflow_vb7ig2sRzbeyyVSY
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 Looks good to me! Incremental review on 5fb163f in 18 seconds
More details
- Looked at
13
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
4
drafted comments based on config settings.
1. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:2
- Draft comment:
New asset import is clear and follows naming conventions. - Reason this comment was not posted:
Confidence changes required:0%
None
2. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:7
- Draft comment:
Mapping added for contactForms; ensure key matches intended template. - Reason this comment was not posted:
Confidence changes required:0%
None
3. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:2
- Draft comment:
Ensure the '@/assets/contact-forms.png' asset exists and is correctly resolved. - Reason this comment was not posted:
Confidence changes required:0%
None
4. skyvern-frontend/src/routes/discover/TemporaryTemplateImages.ts:7
- Draft comment:
Mapping for contact forms is added; verify that the key 'wpid_356554024867713766' is correct and unique. - Reason this comment was not posted:
Confidence changes required:0%
None
Workflow ID: wflow_CQBmQvkZnT42Gs0R
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Important
Add
contactFormsImage
toTEMPORARY_TEMPLATE_IMAGES
inTemporaryTemplateImages.ts
.contactFormsImage
import from@/assets/contact-forms.png
inTemporaryTemplateImages.ts
.wpid_356554024867713766: contactFormsImage
toTEMPORARY_TEMPLATE_IMAGES
inTemporaryTemplateImages.ts
.This description was created by
for 5fb163f. It will automatically update as commits are pushed.